-
Notifications
You must be signed in to change notification settings - Fork 2.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Show msg on save #5269
Show msg on save #5269
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One tiny change please 🙏
Co-authored-by: Alex Beaman <dabeamanator@gmail.com>
Updated |
Hi @aman-atg, I've placed this issue on hold as per this update, as we are prioritising issues related to a feature release scheduled for 9/24. As an apology for the delay, we will add a $100 bonus as a thank you for waiting. |
Main merged! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like this PR is not needed anymore.
oh yea... looks like this PR: #5759 was merged while the original PR was on n6-hold.. |
Aah yep good call fellas. I'd say let's close this, I'll make a note in the issue that you should still get paid 👍 |
Details
#4491 (comment)
Fixed Issues
$ #4491
Tests || QA Steps
Tested On
Screenshots
Web
Web.mov
Mobile Web
mWeb.mov
Desktop
desktop.mov
iOS
ios.mov
Android
Android.mov